Mishina_Violetta_Aleksandrovna#20
Mishina_Violetta_Aleksandrovna#20Violettam88 wants to merge 12 commits intoISUCT:Mishina_Violetta_Aleksandrovnafrom
Conversation
| Console.SetIn(standardIn); | ||
| } | ||
|
|
||
| [Theory] |
There was a problem hiding this comment.
лучше вместо таких сложных тестов поменять код - тесты будут проще и не будет нарушаться принцип единой ответственности
| public string Name | ||
| { | ||
| get | ||
| { | ||
| return name; | ||
| } | ||
|
|
||
| set | ||
| { | ||
| name = value; | ||
| } | ||
| } |
There was a problem hiding this comment.
тут достаточно {get; set:} у вас же нет внутри никакой бизнес логики
| Console.WriteLine($"The country of England has a population of {Population} people and an area of {Square} km^2"); | ||
| } | ||
|
|
||
| public override void ChangePopulation() |
There was a problem hiding this comment.
вот такое вы не протестите в принципе :(
There was a problem hiding this comment.
лучше передавайте параметр извне
saga/Archer (2).cs
Outdated
| @@ -0,0 +1,30 @@ | |||
| namespace CourseApp | |||
There was a problem hiding this comment.
а почему у файла такое странное название?
saga/Archer (2).cs
Outdated
| return "(Лучница) " + Name; | ||
| } | ||
|
|
||
| public override string Ability() |
There was a problem hiding this comment.
а что если способностей много?
|
почему файлы так странно добавлены - тоже будет задание на git |
saga/Game (2).cs
Outdated
| return second; | ||
| } | ||
|
|
||
| if (debaffinf == "Заколдованная пыль") |
There was a problem hiding this comment.
а если таковых будет много?
saga/Logger (2).cs
Outdated
| using System; | ||
| using System.Collections.Generic; | ||
|
|
||
| public static class Logger |
There was a problem hiding this comment.
ну формально - это не логгер :) не надо так (но сейчас не правьте
2 participants
Task_one